Skip to content

Conversation

ianpittwood
Copy link
Contributor

  • Skip calling rstudio-library when chronicleAgent.autoDiscovery=false. Require chronicleAgent.serverAddress be specified instead.
  • Add validation step in rstudio-library.chronicle-agent.serverAddress to check if the lookup returns an empty value.
  • Fail rstudio-library.chronicle-agent.serverAddress render if no server address can be resolved.

@CLAassistant
Copy link

CLAassistant commented Oct 1, 2025

CLA assistant check
All committers have signed the CLA.

@ianpittwood ianpittwood force-pushed the chronicle-server-address-discovery-fixes branch from bc17041 to 73219d1 Compare October 7, 2025 16:24
@matt-urbina matt-urbina marked this pull request as ready for review October 9, 2025 13:55
Comment on lines -5 to +6
version: 0.1.34
appVersion: 0.1.34
version: 0.1.35
appVersion: 0.1.35
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Once this merges we will create a secondary PR for Connect and Workbench to use this version

Copy link
Contributor

@jforest jforest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤘

@matt-urbina matt-urbina merged commit fb57758 into main Oct 9, 2025
6 checks passed
@matt-urbina matt-urbina deleted the chronicle-server-address-discovery-fixes branch October 9, 2025 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants